Codecov Report

Attention: Patch coverage is 63.63636% with 8 lines in your changes missing coverage. Please review.

Project coverage is 98.84%. Comparing base (fc5ba82) to head (6ffe1a8).
Report is 163 commits behind head on master.

@@           Coverage Diff           @@
##           master    #6004   +/-   ##
=======================================
  Coverage   98.84%   98.84%           
=======================================
  Files         396      396           
  Lines       39172    39170    -2     
=======================================
- Hits        38718    38717    -1     
+ Misses        454      453    -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

I hope you have tested it manually.

I hope you have tested it manually.

nope. I trusted t/40-script_load_dump_templates.t

Please provide a commit message. from the reviewers perspective I would like to know the context of the changes. for instance: in what logic or necessity there is change of the function. it also counts as a log for review in the future.

There is no change of function, that should be implied in the message "slightly simplify". I could call it "Refactor slightly", is that better?

I guess mostly the coverage is left here

Please provide a commit message. from the reviewers perspective I would like to know the context of the changes. for instance: in what logic or necessity there is change of the function. it also counts as a log for review in the future.

There is no change of function, that should be implied in the message "slightly simplify". I could call it "Refactor slightly", is that better?

The commit message could be something like Extracting this part because ... bla . it is simple pr, so I dont want to hold you back. if you dont like to add anything let me know and I will approve